Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix ttl tests with start time error by using graph.now() #1478

Merged
merged 1 commit into from
Jun 4, 2021
Merged

fix ttl tests with start time error by using graph.now() #1478

merged 1 commit into from
Jun 4, 2021

Conversation

zhoney
Copy link
Contributor

@zhoney zhoney commented Jun 3, 2021

Change-Id: I27579319df0f732c9a5ee580e47144a63566fb0a

@zhoney
fix ttl tests with start time error by using graph.now()
47f6768 
Change-Id: I27579319df0f732c9a5ee580e47144a63566fb0a
@codecov
Copy link

codecov bot commented Jun 3, 2021 •
edited
Loading

Codecov Report

Merging #1478 (47f6768) into master (461948e) will increase coverage by 0.42%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1478      +/-   ##
============================================
+ Coverage     63.63%   64.05%   +0.42%     
- Complexity     5863     5882      +19     
============================================
  Files           381      381              
  Lines         32190    32190              
  Branches       4506     4506              
============================================
+ Hits          20484    20620     +136     
+ Misses         9608     9463     -145     
- Partials       2098     2107       +9
Impacted Files Coverage Δ
...va/com/baidu/hugegraph/task/ServerInfoManager.java 70.78% <0.00%> (-2.81%) ⬇️
...du/hugegraph/backend/tx/GraphIndexTransaction.java 81.15% <0.00%> (+0.11%) ⬆️
.../baidu/hugegraph/backend/query/ConditionQuery.java 86.40% <0.00%> (+0.40%) ⬆️
...va/com/baidu/hugegraph/backend/cache/RamCache.java 72.41% <0.00%> (+0.57%) ⬆️
.../backend/store/cassandra/CassandraSessionPool.java 57.14% <0.00%> (+1.02%) ⬆️
...egraph/backend/store/cassandra/CassandraShard.java 52.77% <0.00%> (+1.85%) ⬆️
...h/backend/store/scylladb/ScyllaDBTablesWithMV.java 96.03% <0.00%> (+96.03%) ⬆️
...graph/backend/store/scylladb/ScyllaDBFeatures.java 100.00% <0.00%> (+100.00%) ⬆️
.../backend/store/scylladb/ScyllaDBStoreProvider.java 100.00% <0.00%> (+100.00%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 461948e...47f6768. Read the comment docs.

@Linary Linary merged commit 83f6612 into master Jun 4, 2021
@Linary Linary deleted the fix-ttl-start-time-graph-now branch June 4, 2021 09:05
jadepeng pushed a commit to jadepeng/hugegraph that referenced this pull request Jun 25, 2021
@zhoney
fix ttl tests with start time error by using graph.now() (apache#1478)
974ce2a 
Change-Id: I27579319df0f732c9a5ee580e47144a63566fb0a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@javeme javeme javeme approved these changes

@Linary Linary Linary approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@zhoney @Linary @javeme